The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Feb 22, 2026 11:03pm

Greptile Summary

This PR fixes two visual bugs with parallel subflow blocks during workflow execution:

• Active state pulsing: The executor now emits the base block ID (via node.metadata.originalBlockId) instead of the branch-suffixed internal ID, so the UI can correctly match block events to canvas blocks. A ref-counting mechanism ensures a block stays visually active until all parallel branches complete.
• Duration display: The terminal now uses wall-clock time (max(endedAt) - min(startedAt)) for parallel subflow and iteration nodes, instead of incorrectly summing individual branch durations. Loop subflows correctly retain the serial sum.

Key observations:

• The originalBlockId field was already set on all parallel branch nodes during DAG construction (nodes.ts:111) and expansion (parallel-expansion.ts:125), so the fallback ?? node.id safely handles non-parallel nodes.
• The ref-counting logic is duplicated between use-workflow-execution.ts (via buildBlockEventHandlers) and workflow-execution-utils.ts (direct SSE processing for copilot tools). Both paths are consistent but could benefit from a shared helper to prevent future drift.

Confidence Score: 4/5

• This PR is safe to merge — it fixes targeted UI bugs with a well-understood mechanism (ref counting + ID remapping) and no behavioral changes to execution logic.
• The changes are logically sound and well-scoped. The originalBlockId fallback pattern is safe because the field is already populated during DAG construction. The ref-counting logic is straightforward and consistent across both code paths. Score is 4 rather than 5 due to lack of automated tests and duplicated ref-counting logic that could drift.
• Pay attention to workflow-execution-utils.ts — its ref-counting logic duplicates use-workflow-execution.ts and could diverge in future changes.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Executor as BlockExecutor
    participant SSE as SSE Stream
    participant Hook as useWorkflowExecution
    participant RefCount as activeBlockRefCounts
    participant UI as Canvas UI

    Note over Executor: Parallel block A runs in 3 branches<br/>(A__branch-0, A__branch-1, A__branch-2)

    Executor->>SSE: block:started (blockId: A)<br/>via originalBlockId
    SSE->>Hook: event block:started A
    Hook->>RefCount: refCount[A] = 0 → 1
    Hook->>UI: activeBlocks.add(A) → pulse ON

    Executor->>SSE: block:started (blockId: A)
    SSE->>Hook: event block:started A
    Hook->>RefCount: refCount[A] = 1 → 2

    Executor->>SSE: block:started (blockId: A)
    SSE->>Hook: event block:started A
    Hook->>RefCount: refCount[A] = 2 → 3

    Executor->>SSE: block:completed (blockId: A)
    SSE->>Hook: event block:completed A
    Hook->>RefCount: refCount[A] = 3 → 2
    Note over UI: Still pulsing (count > 0)

    Executor->>SSE: block:completed (blockId: A)
    SSE->>Hook: event block:completed A
    Hook->>RefCount: refCount[A] = 2 → 1

    Executor->>SSE: block:completed (blockId: A)
    SSE->>Hook: event block:completed A
    Hook->>RefCount: refCount[A] = 1 → 0
    Hook->>UI: activeBlocks.delete(A) → pulse OFF

_{Last reviewed commit: 04fbe1d}

Addressed in 6b407ee — extracted updateActiveBlockRefCount into workflow-execution-utils.ts as a shared export. Both workflow-execution-utils.ts and use-workflow-execution.ts now call the same function.

@cursor review

@cursor review